-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-19594. Bump Maven 3.9.10 #7760
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: trunk
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice. Thank you.
💔 -1 overall
This message was automatically generated. |
I just found that the Maven installation is different from each OS docker image ... |
never used mvnw; not sure if I'd rush to, but for docker images it makes a lot of sense as a single control point -and makes it trivial for PRs to test new versions, rollback etc |
(!) A patch to the testing environment has been detected. |
(!) A patch to the testing environment has been detected. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
(!) A patch to the testing environment has been detected. |
💔 -1 overall
This message was automatically generated. |
(!) A patch to the testing environment has been detected. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
(!) A patch to the testing environment has been detected. |
💔 -1 overall
This message was automatically generated. |
(!) A patch to the testing environment has been detected. |
(!) A patch to the testing environment has been detected. |
💔 -1 overall
This message was automatically generated. |
(!) A patch to the testing environment has been detected. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
CTest failures look related to junit api assertions now needed on the CP wherever minidfs cluster is created
maybe we should try to cull that check |
@steveloughran do you mean #7782? |
#7782 gets merged, rebase this PR |
(!) A patch to the testing environment has been detected. |
💔 -1 overall
This message was automatically generated. |
Should we continue to process "Dropping support for builds on Centos 7" ? https://lists.apache.org/thread/yj5lx4lmrflyqyfvwmjt74y3jcl65tk0 |
@pan3793 Thanks to Pan for the contribution! Personally, I’m not in a hurry to upgrade to Maven 3.9. I believe this PR should be merged after the unit tests have been fully upgraded. Currently, apart from Spark, I haven’t noticed any other projects upgrading to Maven 3.9, at least not in Ozone and Ratis. I hope we can proceed with JDK 17 support at our own pace. |
Description of PR
Maven prior 3.8 is EOL, bump Maven from 3.6.3 to latest 3.9.10
https://maven.apache.org/docs/3.9.10/release-notes.html
How was this patch tested?
CI.
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?